Fix wrong target for sort-simple-yaml#1113
Conversation
Configure sort-simple-yaml to apply to only yaml files.
|
if you read the docs for this hook you'll understand why the default is the way it is |
|
Where are the docs you're referring to? I had already read the docstrings in https://github.com/pre-commit/pre-commit-hooks/blob/main/pre_commit_hooks/sort_simple_yaml.py and they all talk about YAML files, there is no mention that this hook is meant for other filetypes. |
|
the README is typically the place to look first |
I see what happened: Whoever set this hook up in the monorepo where I hit this did so via I already fixed the regex in our repo (before even submitting this PR). I am still wondering if it makes sense to accept this PR – would including the |
|
no it would make it more difficult in the case that you used this on a file which |
Configure sort-simple-yaml to apply to only yaml files.